-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update webpack-dev-middleware dependency to v3.1.3 #1392
Conversation
package.json
Outdated
@@ -67,7 +67,7 @@ | |||
"spdy": "^3.4.1", | |||
"strip-ansi": "^3.0.0", | |||
"supports-color": "^5.1.0", | |||
"webpack-dev-middleware": "3.1.2", | |||
"webpack-dev-middleware": "^3.1.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for PR, please don't change semver
, it is can break build in future. I.E. remove ^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yikes, didn't even realize I did that! sorry, corrected.
Codecov Report
@@ Coverage Diff @@
## master #1392 +/- ##
=======================================
Coverage 79.14% 79.14%
=======================================
Files 6 6
Lines 494 494
Branches 160 160
=======================================
Hits 391 391
Misses 103 103 Continue to review full report at Codecov.
|
/cc @SpaceK33z |
hi @SpaceK33z any chance this can be merged? |
Sorry for the delay. |
cheers! no worries at all, thank you! |
Motivation / Use-Case
Spaces in the
outputPath
are not being handled correctly on Windows.This change addresses the regression which introduced the bug and is being tracked in #1375
The only change made is that the
package.json
has been updated to reference v3.1.3 of webpack-dev-middleware.The issue was corrected here:
webpack/webpack-dev-middleware#297